Skip to content

loadout-lab: 0.3.2#14014

Open
ajkatz wants to merge 1 commit into
runelite:masterfrom
ajkatz:loadout-lab-0.3.1
Open

loadout-lab: 0.3.2#14014
ajkatz wants to merge 1 commit into
runelite:masterfrom
ajkatz:loadout-lab-0.3.1

Conversation

@ajkatz

@ajkatz ajkatz commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Version bump 0.3.0 → 0.3.2, now that the AKAddons ownership move (#14013) has merged. This build drops java.lang.Runtime entirely (see below), so the diff here is a single commit= line.

Highlights since 0.3.0: curated monster groups that expand into one multi-mob answer (raids, wildy bosses, phases with per-style immunities); a cross-style carried-swaps kit with a trip inventory view; wilderness risk-cap fixes; plus a token-diet pass and spec value-over-replacement modelling.

Re: the Runtime.getRuntime() note — removed. The multi-mob roster fan-out pool was sizing itself with Runtime.getRuntime().availableProcessors(); it now uses a fixed pool of 4 min-priority daemon threads, so there is no java.lang.Runtime reference anywhere in the source.

🤖 Generated with Claude Code

@runelite-github-app

runelite-github-app Bot commented Jul 19, 2026

Copy link
Copy Markdown

@runelite-github-app

runelite-github-app Bot commented Jul 19, 2026

Copy link
Copy Markdown

This plugin requires a review from a Plugin Hub maintainer. The reviewer will request any additional changes if needed.


Internal use only: Reviewer details Maintainer details

@pajlada

pajlada commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Please perform the repository change first @ajkatz, otherwise the diff viewing gets messed up

@pajlada pajlada added the waiting for author waiting for the pr author to make changes or respond to questions label Jul 19, 2026
@ajkatz

ajkatz commented Jul 19, 2026

Copy link
Copy Markdown
Contributor Author

Understood - I've reopened #14013 (the ownership move, manifest-only, commit unchanged) and it should go first. This PR is parked until that merges; I'll rebase it onto master afterwards so the diff here is a clean one-line commit= change. Sorry for the churn.

@runelite-github-app runelite-github-app Bot removed the waiting for author waiting for the pr author to make changes or respond to questions label Jul 19, 2026
@ajkatz

ajkatz commented Jul 19, 2026

Copy link
Copy Markdown
Contributor Author

Please perform the repository change first @ajkatz, otherwise the diff viewing gets messed up

@pajlada #14013

@ajkatz
ajkatz force-pushed the loadout-lab-0.3.1 branch from 3101294 to fb51ef5 Compare July 20, 2026 00:03
@ajkatz

ajkatz commented Jul 20, 2026

Copy link
Copy Markdown
Contributor Author

Rebased onto master now that #14013 (the AKAddons ownership move) has merged. The diff here is now a single commit= line - repository= and authors= already match master. Ready for review, thanks @pajlada.

@ajkatz
ajkatz force-pushed the loadout-lab-0.3.1 branch from fb51ef5 to 6469c51 Compare July 20, 2026 06:53
@ajkatz ajkatz changed the title loadout-lab: 0.3.1 loadout-lab: 0.3.2 Jul 20, 2026
@ajkatz
ajkatz force-pushed the loadout-lab-0.3.1 branch from 6469c51 to 89bb186 Compare July 21, 2026 18:05
@ajkatz

ajkatz commented Jul 21, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for the catch @pajlada - the Runtime.getRuntime() use is gone. It was sizing the multi-mob roster thread pool via availableProcessors(); it's now a fixed pool of 4 min-priority daemon threads, so the source no longer references java.lang.Runtime at all. I also rolled the pin forward to 0.3.2 (current head) so the fix ships on the latest build rather than a superseded 0.3.1 - the diff here stays a single commit= line.

@ajkatz
ajkatz force-pushed the loadout-lab-0.3.1 branch from 89bb186 to d9daba0 Compare July 21, 2026 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants